Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Pathway Plan - Objectives and Tasks #145

Closed
wants to merge 45 commits into from
Closed

Pathway Plan - Objectives and Tasks #145

wants to merge 45 commits into from

Conversation

samgibsonmoj
Copy link
Contributor

No description provided.

@samgibsonmoj
WIP: Pathway Plan / Objectives screen
db4447c
@samgibsonmoj
Objective task completion dates
a06991d
@samgibsonmoj
Objective/Task data modelling and migrations
5730d19
@samgibsonmoj
WIP: Add tasks on a month basis, instead of specific dates
45ea502
@samgibsonmoj
Objective/Task dto's and command handlers
a2854f8
@samgibsonmoj
Merge remote-tracking branch 'origin/main' into feature/objectives
d93bf2e
@samgibsonmoj
Merge remote-tracking branch 'origin/main' into feature/objectives
0660e2b
@samgibsonmoj
Merge remote-tracking branch 'origin/main' into feature/objectives
d67a0ae
@samgibsonmoj
Objective/task create and update commands
215fbaf
@samgibsonmoj
Merge remote-tracking branch 'origin/main' into feature/objectives
ce4d2d7
@samgibsonmoj
Fix styling and add close button for dialogs
28759bb
@samgibsonmoj
Remove 'close' option for task and allow selection of completion reason
2b4d461
@samgibsonmoj
Objective sorting
7e9ee47
@samgibsonmoj
Add ability to rename objectives
db17891
@samgibsonmoj
Objective status & review
1af4f65
@samgibsonmoj
Cleanup pathway plan + separate objective/task components
3dad9d2
@samgibsonmoj
Re-add real-time status to objective
17a5639
@samgibsonmoj
WIP - Pathway Plan reviews
926d93a
@samgibsonmoj
CatsComponentBase - OnUpdate propagation
551fdfe
@samgibsonmoj
Add pathway plan
2ff9b02
@samgibsonmoj
Pathway plan review command
418e15a
@samgibsonmoj
Pathway plan review functionality + case summary information
79b3c2e
@samgibsonmoj
Remove review action from case summary
5086eae
@samgibsonmoj
Fix typos and ensure immutability in objectives
b12a338
@samgibsonmoj
Identifiers for objectives/tasks
818436e
@samgibsonmoj
Fix typo for regex validator name
ce4bd8f
@samgibsonmoj
Restrict allowed characters for objective/task titles
7ff4db6
@samgibsonmoj
Add expansion dialogs for displaying completion reason
6cb45d8
@samgibsonmoj
Merge remote-tracking branch 'origin/main' into feature/objectives
7de98d3
@samgibsonmoj
Resolve merge conflict
79a6e15
PaulCooperWorkJustice
PaulCooperWorkJustice requested changes Aug 19, 2024
View reviewed changes
Copy link
Contributor

@PaulCooperWorkJustice PaulCooperWorkJustice left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

hi can you hide icons instead of disabling them :)

src/Server.UI/Pages/Objectives/Objective.razor Outdated Show resolved Hide resolved
PaulCooperWorkJustice
PaulCooperWorkJustice reviewed Aug 19, 2024
View reviewed changes
Copy link
Contributor

@PaulCooperWorkJustice PaulCooperWorkJustice left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I've checked code and looks good. Waiting for Carl to Approve.

I've run locally and tested it locally and it all works.

@samgibsonmoj
Copy link
Contributor Author

Made changes as per @PaulCooperWorkJustice's suggestions

@carlsixsmith-moj
Copy link
Contributor

Changing to merge to develop branch

@samgibsonmoj samgibsonmoj deleted the feature/objectives branch August 21, 2024 09:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@PaulCooperWorkJustice PaulCooperWorkJustice PaulCooperWorkJustice requested changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@samgibsonmoj @carlsixsmith-moj @PaulCooperWorkJustice